Skip to content

fix(helm): add Prometheus ServiceMonitor support for operator metrics - #36

Merged
pierinho13 merged 1 commit into
mainfrom
fixing_service_monitor
Aug 9, 2026
Merged

fix(helm): add Prometheus ServiceMonitor support for operator metrics#36
pierinho13 merged 1 commit into
mainfrom
fixing_service_monitor

Conversation

@pierinho13

Copy link
Copy Markdown
Owner

Summary

Add optional Prometheus Operator ServiceMonitor support to the Helm chart so the GitHub Platform Operator metrics endpoint can be scraped without requiring users to create monitoring resources manually.

Changes

  • Add optional metrics.serviceMonitor Helm configuration.
  • Add a ServiceMonitor for the operator metrics service.
  • Support configurable scrape interval and timeout.
  • Support additional labels and annotations for Prometheus Operator discovery.
  • Support an optional Prometheus ServiceAccount for accessing the secured /metrics endpoint.
  • Create the required metrics-reader binding when a Prometheus ServiceAccount is configured.
  • Extend values.schema.json with the new monitoring configuration.
  • Document the ServiceMonitor configuration in the Helm chart README.

Example

metrics:
  enabled: true
  secure: true
  port: 8443

  serviceMonitor:
    enabled: true
    namespace: monitoring

    additionalLabels:
      release: kube-prometheus-stack

    interval: 30s
    scrapeTimeout: 10s

    prometheusServiceAccount:
      name: kube-prometheus-stack-prometheus
      namespace: monitoring

@pierinho13
pierinho13 merged commit 91ca2d2 into main Aug 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant